Skip to content
This repository has been archived by the owner on Jun 25, 2020. It is now read-only.

Dynamic front page feature row #94

Merged
6 commits merged into from
Apr 14, 2020
Merged

Dynamic front page feature row #94

6 commits merged into from
Apr 14, 2020

Conversation

Spirosvw
Copy link
Collaborator

@Spirosvw Spirosvw commented Apr 6, 2020

Σχετικό Issue

closes #78

Προτεινόμενες Αλλαγές

-- Αλλαγή 1
-Δημιουργία φακέλου _includes και dynamic.html για την υλοποίηση της δυναμικής αλλαγής του περιεχομένου στην αρχική σελίδα

-- Αλλαγή 2
-Επεξεργασία του αρχείου index.html για την χρήση του dynamic.html για την εμφάνιση δυναμικού περιεχομένου

--Τελικό Αποτέλεσμα
Πλέον στην αρχική σελίδα εμφανίζονται με τυχαιότητα μία από τις ανακοινώσεις, ένα από τα projects και ένα από τα εργαστήρια. Το καθένα περιλαμβάνει ένα τίτλο και ένα μικρό περιλιπτικό κείμενο. Κλικάροντας στον τίτλο ο χρήστης οδηγείται στην σελίδα του αντίστοιχου περιεχομένου (ανακοίνωση ΧΧ, project ΧΧ ...) και κλικάροντας το "Learn More" κάτω από κάθε κατηγορία οδηγεί αντίστοιχα στην σελίδα με τα posts, στην σελίδα με τα projects ή στην σελίδα με τα εργαστήρια.

Ενδεικτικό link: https://sharp-borg-3fc7ea.netlify.com/

Υπενθυμίσεις

  • Έχω ανοίξει από πριν issue για τον καλό συντονισμό του project, το οποίο έχει πάρει το πράσινο φως με την αντίστοιχη ετικέτα
  • Έχω δημιουργήσει branch για τις αλλαγές

@epidrome epidrome assigned ghost Apr 8, 2020
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Υπάρχει ένα μικρό regression που μπορεί να αλλάξει εύκολα.

index.html Outdated
Comment on lines 8 to 9
cta_label: "<i class='fa fa-download'></i> Οδηγός Σπουδών 2016-2017"
cta_url: "http://di.ionio.gr/wp-content/uploads/2016/09/OS_2016-2017-final.pdf"
Copy link

@ghost ghost Apr 8, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Γιατί υπάρχει αυτή η αλλαγή εδώ που είναι fallback σε πιο παλιό link? Είναι λάθος που ξέφυγε.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Πρεπει να ειναι επειδη χρησιμοποιησα παλιο version του html

@Spirosvw
Copy link
Collaborator Author

Spirosvw commented Apr 9, 2020

Υπάρχει ένα μικρό regression που μπορεί να αλλάξει εύκολα.

Χρησιμοποιήσα παλιότερη version του html. Διόρθωσα το λάθος

@Spirosvw Spirosvw requested a review from a user April 9, 2020 10:30
@epidrome epidrome requested review from epidrome and removed request for epidrome April 13, 2020 13:43
@epidrome
Copy link
Member

μπράβο, φαίνεται ωραιά
το μόνο σημαντικό σχόλιο που έχω είναι
ότι το ινκλουντ ταιριάζει καλύτερα στο θέμα
για την ώρα, αν συμφωνούν και οι αξιολογητές
ας το κάνουμε δεκτό και μετά το διαγράφεις-στέλνεις
στο μινιμαλ-ιονιο

@p15zerv @GIANNIS-AGGELIS
αν θέλετε να το δείτε με προτεραιότητα
ώστε να προχωρήσει ο @Spirosvw στο
επόμενο παραδοτέο του

Copy link
Collaborator

@p15zerv p15zerv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Κι εμένα μια χαρά μου φαίνεται. Συμφωνώ ότι το dynamic.html μάλλον είναι καλύτερα να πάει στο theme, αλλά για τώρα μπορεί να γίνει δεκτό κι ας αλλάξει αργότερα.

Copy link
Collaborator

@GIANNIS-AGGELIS GIANNIS-AGGELIS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Μια χαρά μου φαίνονται οι αλλαγές που έκανε ο @Spirosvw . Θα συμφωνήσω με τον @p15zerv για τι μεταφορά του αρχείου αργότερα.

@epidrome
Copy link
Member

@constantinexisc @Spirosvw

ούτε τοπικά, ούτε στο λινκ του αποθετηρίου

δεν βλέπω τις δυναμικές αναρτήσεις???

@ghost
Copy link

ghost commented Apr 16, 2020

@epidrome

Να εξηγήσω, η δυναμικότητα έιναι βασισμένη στη μεταβλητή site.time, αφού το jekyll είναι static site generator δεν μπορεί να έχει κάποιο άλλο είδος δυναμικότητας ή τυχαιότητας αφότου έχει "χτισει" τις σελίδες και τις κάνει serve.

Κάθε φορά που θα γίνεται καινούργιο jekyll serve το site.time θα ενημερώνεται και θα δίνει κάποια άλλα post στην αρχική.

@epidrome
Copy link
Member

@constantinexisc
ευχαριστώ! γνωρίζω πως δουλεύει, αφού η υλοποίηση βασίζεται στην δικιά μου από το σαιτ του βιβλίου

η απορία ήταν δικό μου λάθος με κασινγκ της παλιάς σελίδας στον σαφάρι, συγνώμη!

@provopoulos provopoulos mentioned this pull request Apr 16, 2020
2 tasks
@dimpram
Copy link
Collaborator

dimpram commented Apr 19, 2020

Συγνώμη για το αργοπορημένο σχόλιο, δεν είχα παρατηρήσει την εξέληξη αυτού του issue... Υπάρχουν μερικά προβλήματα με αυτό το PR κατά την γνώμη μου και του @provopoulos. Πρώτον παρατηρήθηκε οτι πλέον το mobile view στο homepage έχει καταστραφεί καθώς τα κουμπία "Learn more" δεν είναι στις σωστές θέσεις τους. Επιπλέον, υπάρχει άλλο ένα θέμα με την διπλή γραμμή ακριβώς από κάτω από το feature row το οποίο υποψιάζομαι έχέι να κάνει με κάποιο λανθασμένο div nesting. Τέλος με το να υπάρχει το include σε αυτό το αποθετήριο δεν νομίζω οτι αποτελεί ολοκληρομένη συνεισφορά καθώς είχαμε δώσει ιδιαίτερη έμφαση στο γεγονός οτι έχουμε το minimal-ionio theme σε ξεχωριστό αποθετήριο μαζί με τα includes του και τα συναφή. Αν ο λόγος που συνέβη αυτό βέβαια είναι για να διευκολυνθέι το minimal-ionio upgrade τότε δεκτό αλλά θα πρέπει να ξαναπροσαρμωστεί αμέσως.

Ωστόσο δεν θα ήταν πιο σωστό να ήταν η πιο πρόσφατη ανακοίνωση και όχι κάποια τυχαία;

Ξανά, συγχωρέστε με για το αργοπορημένο σχόλιο αλλά ήθελα να μοιραστώ τις σκέψεις μου και του @provopoulos.

@provopoulos
Copy link
Collaborator

For clarity:
BEFORE
AFTER
Επειδή χρειάστηκε να κάνω αλλαγές σε σχετικό issue: #83 (comment), έχω μεταφέρει το dynamic.html στο αποθετήριο του θέματος και οι αλλαγές που έχω κάνει στο index.html επιλύουν (νομίζω) το ζήτημα με το DIV_NESTING (ευχαριστώ @JimDragon) ωστόσο τα Learn More εξακολουθούν να μην εμφανίζονται σωστά.

@provopoulos
Copy link
Collaborator

Σε αυτά που είπε ο Δημήτρης θα ήθελα να προσθέσω μια πιθανή μελλοντική βελτίωση. Να υπάρχει κάποιο όριο, πιθανόν, εμφάνισης λέξεων ώστε να μην υπάρχει ασυμφωνία ευθυγράμμισης των στηλών, π.χ. https://sharp-borg-3fc7ea.netlify.com/
Το Twitter module: #83, έχει συγκεκριμένες (fixed) διαστάσεις data-width="" data-height="" οπότε θα ήταν καλό οι υπόλοιπες στήλες να ευθυγραμμίζονται ως προς αυτό κι όχι το αντίθετο.

@epidrome epidrome changed the title Dynaminc data Dynamic front page feature row Apr 22, 2020
@epidrome
Copy link
Member

@JimDragon μπράβο σου για την αναλυτική περιγραφή, η οποία όμως για να είναι χρήσιμη στο έργο θα πρέπει να σπάσει σε επιμέρους τμήματα τα οποία θα γίνουν νέα θέματα σε συντονισμό με τον @provopoulos που φαίνεται να έχει βρει λύση για κάποια και με ανάθεση σε συντελεστές που θα μπορούσαν να βοηθήσουν

προφανώς δεν έχει νόημα να τα συζητάμε εδώ, γιατί είναι σαν "να κλαίμε πάνω από το χυμένο γάλα", ας δώσουμε λύση καλύτερα!

This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Δυναμικό περιεχόμενο στην αρχική σελίδα
6 participants